GtkApplicationWindow: the help_overlay is nullable

GtkApplicationWindow: Fix typos

Adwaita: Fix window shadow size

Ensure that the shadow extents don't change as we go to backdrop,
to prevent windows from jumping.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explicit state variables

Fold it into its single user.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explict state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid an explicit state variable

Using the state of the context makes it more obviously correct.

listbox: Cosmetic cleanup

We don't cast these arguments anywhere else, so lets not do here
either.

window: Don't invalidate cssnode during get_preferred_width()

Getting the shadow width must not call gtk_style_context_set_state()
because that will invalidate the node and cause a style-updated emission
which can cause gtk_widget_queue_resize() calls.

And calling queue_resize() from get_preferred_size() essentially means
the size is permanently invalid because you invalidate it while
querying it.

This causes flickering of windows when going from/to backdrop state. To
avoid this we either need to fix the theme to not have different shadow
sizes in those cases or we need to ensure the window doesn't flicker in
the first place.

Us ... instead of …  for zh_CN

Updated Portuguese translation

GtkShortcutsShortcut: Add missing word

GtkGrid: Remove invalid return annotation

gtkimcontextsimple.c: Use X11_DATA_PREFIX only on X11/Wayland

Only use the hard-coded build-time path given by X11_PREFIX on X11 and
Wayland where a X11 package is normally available.  On other platforms,
get the datadir of the running system and mimic the behavior by
constructing the path dynamically.  This avoids hardcoding the path for
searching for compose tables where we want to have relocatability.

This fixes the build on Windows/MSVC as well, where we don't normally have
any X11 packages available.

https://bugzilla.gnome.org/show_bug.cgi?id=757984

check menu item: Fix a typo

It is check, not checl. Pointed out by Giovanni Campagna in
https://bugzilla.gnome.org/show_bug.cgi?id=757968

Adwaita: Fix styling of selected rows in treeviews

This broke in some of the recent selection handling changes.

filechooser: ignore double clicks on the selected item

Since we are now interpreting button press events and
make our own double-click determination, we should not
handle double-click events that are generated by GDK.

https://bugzilla.gnome.org/show_bug.cgi?id=757950

win32: do not use g_clear_pointer also on the clipboard window

A follow up on the previous patch. We should use DestroyWindow
directly since it has a different calling convention than
the expected callback for g_clear_pointer

win32: do not use g_clear_pointer to destroy the window

DestroyWindow expects a different calling convenction so
we endup getting an error at runtime

Update testlevelbar

This test needs some small updates after the recent changes
to levelbar styling.

notebook: Chain up in direction_changed

We do this everywhere else.

Updated Hebrew translation

HighContrast: Update levelbar styling

Adapt to changes in the previous commit.

Adwaita: Update levelbar styling

Adapt to the changes in the previous commit. In particular, fix
the handling of low and high offsets. Anything below the low offset
gets warning color, anything below high gets selected background,
and anything below the new full offset gets success color.

levelbar: Use more CSS nodes

Create as many CSS nodes as we're rendering blocks on the screen.

textbuffer: nicer get_iter functions, without return values

Avoid crashes when passing an invalid location to a
gtk_text_buffer_get_iter_at_*() function.

A first attempt added boolean return values to know if @iter has been set to
the exact location, but it breaks Python and JS bindings because the out
parameter is already a return value in those languages.

Unit tests are added.

https://bugzilla.gnome.org/show_bug.cgi?id=735341

win32: destroy clipboard notification window on dispose

The clipboard uses a hidden window to get some specific events.
The window was created but never destroyed on dispose.

update zh_CN translation

win32: chain up on display dispose

Revert "textbuffer: nicer get_iter functions"

This reverts commit a9a1c00cc9255ee9a7f73695f69abe172cde7bfa.

Unfortunately, adding the boolean return broke both the python
and javascript bindings, since they now return a tuple consisting
of the boolean and the out argument.

gtktestutils.c: Ensure variables are declared at top of block

Fixes build on C89 compilers...

MSVC builds: Use /opt:ref for release builds

glib-compile-resources have been updated to ensure that the symbols
generated are referred to, so that they will not be optimized out by the
linker in release builds.  We can change from /opt:noref to /opt:ref,
which should improve optimization a bit.

MSVC 2010+ builds: Do not explicitly use /LTCG

This partially reverts de16a4e.

As we now ensure that items using GResources and GConstructors are always
referenced so that the linker does not optimize them out in a default
Release build, we no longer need to enforce the use of /LTCG, so
/LTCG:incremental will work as well.

Add gtk_native_dialog_destroy()

Its very easy to get extra references to the NativeDialog so that
when you release your last reference any visible dialog is not
hidden. We handle this by adding a destroy method similar to how
you destroy regular toplevels.

GtkNativeDialog: Fix modal handling in gtk_native_dialog_run()

tooltipwindow: Use a builder UI file

It's not a hugely complicated file, but it's easier to deal with some of
the details of tooltip windows styling if we have a UI file to edit,
instead of source code.

Add autocleanups for GtkShortcutsWindow

Remove introspection scanner guards around autocleanups

They are not needed.

gtklevelbar: Fix typo

HighContrast: Update levelbar styling

Adapt to the changes in the previous release.

Adwaita: Update levelbar rendering

Adapt to the changes in the previous commit.

levelbar: Port to use CSS nodes

Use element names levelbar, trough, block, and some style
classes on the block nodes: .discrete, .continuous, .empty,
.level-low, etc.

Improve testlevelbar

Add a switch to toggle between discrete and continuous modes.

text util: Avoid creating transient nodes

We don't need to add .view - its already added, and shouldn't
really be needed in the first place.

HighContrast: Update textview selection styling

Use the selection element name.

Adwaita: Update textview selection styling

Use the selection name.

text view: Improve text rendering

Use the text CSS node for rendering text, and the selection node
for rendering selected text, avoid gtk_style_context_save, update
states of all CSS nodes, and use the proper states when querying
style properties.

textview: Use a CSS node for selection rendering

Use a CSS node with name selection, like we do for entries
and labels. Unlike those widgets, we currently don't user
gtk_render_background, but just use the background color.
That will require more effort.

cell view: Add an element name

The element name is...cellview.

HighContrast: Update label selection styling

Adapt to changes in the previous commit.

Adwaita: Update label selection styling

Adapt to the changes in the previous commit.

label: Use a CSS node for selection rendering

This adds new possibilities for themes to render the selection
in fancy ways.

entry: Simplify selection rendering

Calling gtk_render_background for each rectangle in the region
leads to suboptimal and sometimes weird results. Getting this
right requires more work in Pango first. Go back to just rendering
a single background, and clip it to the selection region. This
matches what GtkLabel does.

inspector: Don't grow horizontally

Labels in the object tree are application data. Ellipsize them
to avoid growing extremely wide if the data is bogus.

inspector: fix a typo

HighContrast: Update entry selection styling

Adapt to changes in the previous commit.

Adwaita: Update entry selection rendering

Adapt to the changes in the previous commit.

entry: Use a CSS node for selection rendering

This adds new possibilities for themes to render the selection
in fancy ways.

Adwaita: Update tooltip styling more

Now that tooltips are csd again, the CSS wasn't quite working
anymore.

gtkwindow: css offset for toplevel only

At the time gtk_window_move() or gtk_window_resize() get called, there
is no way to predict if a popup window will actually draw its shadow, so
applying an offset in this case may end up with a wrong size or
positioning for such windows.

Changing the logic in gtk_window_should_use_csd() as previously done to
address that issue will cause some other breakage as popup windows may
not draw a shadow but still need CSD.

So best is to actually apply client side decorations offset for regular,
top level windows only. This is actually a lot simpler and safer and
less likely to cause additional breakage.

Bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=756618

Adwaita: update tooltip styling

Adapt to the changes in the previous commit. Note that tooltip
appearance is currently affected by tooltips having lost their
csd nature, due to a regression.

Move GtkAccelMap autocleanup to the class header

Recursively move autocleanups to their header files

Start from GtkAccelLabel and walk the tree downwards.

Move GtkAboutDialog autocleanup to the class header

Move GtkDialog autocleanup to the class header

Start recursively moving autocleanups macros to their headers

* Cover letter

Having a single header file for all autocleanups definitions was a
reasonable stop-gap measure, but now GTK+ is starting to use G_DECLARE_*
macros. This means that every class using a G_DECLARE_* macro will need
to include "gtk.h" to avoid compiler warnings, which is not acceptable.

By moving the G_DEFINE_AUTO* use to the header that defines the type we
allow using the G_DECLARE_* macros without sacrificing the ability to
include only the needed files when deriving from a class.

* Commit

This commit changes all includes relative to GtkWindow to define their
own autocleanup macros.

Only include the necessary headers in GtkPlacesViewRow

When compiling inside GTK we also use the GTK_COMPILATION guard, as the
GtkPlacesView and GtkPlacesViewRow widgets are shared with Nautilus.

Move GtkListBox g_autoptr macros to the class header

This allows GtkPlacesViewRow to use G_DECLARE_FINAL_TYPE without having
to include gtk.h.

Remove an unused variable declaration

And the compiler warning that comes with it.

Move GtkWindow g_autoptr macros in the class header

Otherwise we won't be able to use G_DECLARE_* macros internally to GTK
without generating warnings, or without including gtk.h itself.

docs: Ignore GtkTooltipWindow's private header file

Fix an oversight

We must set the construct-only "type" to GTK_WINDOW_POPUP, otherwise
this ends upo being a regular toplevel.

Use CSS node name for GtkTooltipWindow

Instead of a style class.

Add GtkTooltipWindow private sub-class

GtkTooltip does a lot of set up on the GtkWindow it uses internally. We
should move that code to a separate class to keep it contained.

print dialog: Avoid gtk_style_context_save

There was one more call to gtk_style_context_save here that I
overlooked. We can get rid of it by just setting the element name
on the drawing area.

flowbox: Add a diagram to CSS documentation

iconview: Add a diagram to CSS documentation

treeview: Add a diagram to CSS documentation

HighContrast: Update treeview rubberband styling

Adapt to the changes in the previous commit.

Adwaita: Update treeview rubberband styling

Adapt to the changes in the previous commit.

treeview: Use a CSS node for rubberband drawing

Use a CSS node with name rubberband to draw the rubberband
selection.

TextView: Use saner coordinate space in draw_layer.

When I added the draw_layer vfunc it accidentally got passed a cairo_t
that was configured with to draw in the viewport coordinate space (rather
than the buffer coordinate space). This makes things unnecessary complex,
because you have to convert between the two.

The pixel cache is shared between the text and the layers, so there is
no way to use draw_layer to get a stationary overlay effect. Thus it makes
much more sense for the draw_layer vfunc to draw in the buffer space.

Just changing this would break ABI for existing code, so this is fixed
by adding new layer types and deprecating the old ones.

Also, we use the new layer types to fix gtk3-widget-factory.

https://bugzilla.gnome.org/show_bug.cgi?id=757856

Forgotten file

This change belongs to the iconview rubberband changes.

HighContrast: Update flowbox rubberband styling

Adapt to the changes in the previous commit.

Adwaita: Update flowbox rubberband styling

Adapt to the changes in the previous commit.

flowbox: Use a CSS node for rubberband drawing

Use a CSS node with name rubberband to draw the rubberband
selection.

HighContrast: Update rubberband styling

Adapt to the changes in the previous commit.

Adwaita: Update rubberband styling

Adapt to the changes in the previous commit.

iconview: Use a CSS node for rubberband drawing

Use a CSS node with name rubberband to draw the rubberband
selection.

HighContrast: Update text handle styling

Adapt to the changes in the previous release.

Adwaita: Update text handle styling

Adapt to the changes in the previous commit.

Document text handle styling

Since GtkTextHandle is private, document text handle style
classes in the GtkEntry and GtkTextView documentation.

text handle: Port to CSS nodes

Use cursor-handle as the element name for the CSS node that
is used to render text the selection handles.